Skip to content

Hw 4#4

Merged
Alex777Russia merged 11 commits intomasterfrom
hw_4
Jan 26, 2026
Merged

Hw 4#4
Alex777Russia merged 11 commits intomasterfrom
hw_4

Conversation

@Alex777Russia
Copy link
Owner

Сделал ДЗ № 4

Алексей Шевченко added 2 commits January 15, 2026 20:11
Copy link
Collaborator

@sh-alanova sh-alanova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Что сделано хорошо:

  • Отлично проработан roadmap на будущее (это, как я написала ниже, можно вынести в отдельный раздел «План масштабирования»)

  • Чётко описаны и прописаны integration points, безопасность как системы, так и данных, также риски и неопределённости -- это полезно для MVP

НО раздел «4. Внедрение» требует доработки.


### 4. Внедрение для production систем

**Важное примечание:**
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Раздел 4 описывает архитектуру гипотетической production-системы (v2.0+), но это, всё-таки, не соответствует требованиям задания, которое просит описать инфраструктуру текущего решения.

То есть нужно всё же:

  1. написать описание реального стека MVP -- решения, которое реализовано и работает сейчас (н-р, Streamlit, Qdrant in-memory, запуск через docker-compose или python app.py),

  2. добавить реальную инфраструктуру MVP, с описанием того, что фактически реализовано (н-р, к описанию локальной инфраструктуры относястся: ОС: Linux / macOS / Windows, версия Python, хранилище - локальный Qdrant in-memory или ChromaDB, ... etc.)

  3. провести простое нагрузочное тестирование на данных (на небольшом сете - 10-50 запросов, или на реальных данных, которые имеются, или на данных, которые были описаны в разделе 2.3) и зафиксировать цифры

  4. результаты замеров (н-р, latency, throughput, RAM)

  5. можно также добавить какие-то скриншоты, таблицы или графики нагрузочного тестирования

Описанный текст в данной версии документа рекомендую перенести как отдельный подраздел (н-р, «План масштабирования»), чтобы не смешивать настоящее и будущее.

@itwastony
Copy link
Collaborator

6/10, не нашел app.py, контейнер не соберется, нет api, отсутствует нагрузочное тестирование

@Alex777Russia
Copy link
Owner Author

@itwastony, привет!

Согласен, поправил:

@itwastony
Copy link
Collaborator

@itwastony, привет!

Согласен, поправил:

10/10

@Alex777Russia Alex777Russia merged commit 868d7fe into master Jan 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants